Skip to content

chore!: Qdrant - remove init_from init parameter for compatibility with qdrant-client==1.16.0#2531

Merged
anakin87 merged 2 commits into
mainfrom
qdrant-rm-init-from
Nov 18, 2025
Merged

chore!: Qdrant - remove init_from init parameter for compatibility with qdrant-client==1.16.0#2531
anakin87 merged 2 commits into
mainfrom
qdrant-rm-init-from

Conversation

@anakin87
Copy link
Copy Markdown
Member

@anakin87 anakin87 commented Nov 18, 2025

Related Issues

Failing tests: https://github.com/deepset-ai/haystack-core-integrations/actions/runs/19449131010

The new release of qdrant-client (1.16.0) removes deprecated init_from init parameter from create_collection: https://github.com/qdrant/qdrant-client/releases/tag/v1.16.0

Proposed Changes:

  • remove init_from init parameter from QdrantDocumentStore.__init__
  • solve a typing issue: grpc.PointId is no longer accessible for type checkers but exists at runtime

How did you test it?

CI

Notes for the reviewer

This is done for compatibility with qdrant-client==1.16.0 but is a breaking change. I'll release a new major version.

Checklist

@github-actions github-actions Bot added integration:qdrant type:documentation Improvements or additions to documentation labels Nov 18, 2025
@anakin87 anakin87 marked this pull request as ready for review November 18, 2025 09:04
@anakin87 anakin87 requested a review from a team as a code owner November 18, 2025 09:04
@anakin87 anakin87 requested review from mpangrazzi and removed request for a team November 18, 2025 09:04
@anakin87 anakin87 self-assigned this Nov 18, 2025
@anakin87 anakin87 changed the title chore: Qdrant - remove init_from init parameter for compatibility with qdrant-client==1.16.0 chore!: Qdrant - remove init_from init parameter for compatibility with qdrant-client==1.16.0 Nov 18, 2025
Copy link
Copy Markdown
Contributor

@mpangrazzi mpangrazzi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good!

@anakin87 anakin87 merged commit 8e54549 into main Nov 18, 2025
10 checks passed
@anakin87 anakin87 deleted the qdrant-rm-init-from branch November 18, 2025 17:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

integration:qdrant type:documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants